Conversation
🦋 Changeset detectedLatest commit: 0947efa The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR 💥 An error occurred when fetching the changed packages and changesets in this PR |
✅ Deploy Preview for starlight-latest-version ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
WalkthroughUpdated dependency and engine constraints across docs and packages: Astro and Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/package.json`:
- Around line 29-36: The starlight-latest-version workspace plugin declares a
Starlight peer but lacks an explicit Astro peer while the docs app uses Astro 6;
update the plugin package manifest (packages/starlight-latest-version
package.json) to add a peerDependencies entry for "astro" (e.g., "astro": "^6"
or an appropriate range), run the plugin against Astro 6 to verify
compatibility, and if any dev or test tooling is pinned to Astro 5
(devDependencies or local test env pinned to 5.16.5), update those pins to a
matching 6.x version or run compatibility fixes; alternatively, if you cannot
confirm compatibility, revert the docs app Astro version to the supported Astro
5 range instead.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 8881f6f7-f694-4fcc-b8c8-a2cdc896a240
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (1)
docs/package.json
Needs to wait for withastro/astro#15753
Needs to wait for withastro/astro#15888 as well
Summary by CodeRabbit
Chores
Breaking Changes